-
Notifications
You must be signed in to change notification settings - Fork 248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CS2113-W13-1] DoctorDuke #54
Open
Jeraldchen
wants to merge
390
commits into
nus-cs2113-AY2223S2:master
Choose a base branch
from
AY2223S2-CS2113-W13-1:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[CS2113-W13-1] DoctorDuke #54
Jeraldchen
wants to merge
390
commits into
nus-cs2113-AY2223S2:master
from
AY2223S2-CS2113-W13-1:master
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Jeraldchen user guide
Jeraldchen logging
Added documentation to important methods and classes involving diagnosis. Changed author comments to Javadoc @author tags. Remove commented test code for diagnosis again. Removed some redundancies.
Added an assertion in Diagnosis.java to prevent a divide by zero in the event the programmer created an Illness child with no symptoms. Added logging to signal when diagnosis was ran, and what is the comparison value to each of the possible illnesses. Fixed a typo on authorship @author tag.
Branch brannon 202303071516
Enable assertion in build.gradle
Jeraldchen multiple inputs
Prevent duplicate passwords and hash passwords
Added more medications for illnesses.
Allow user to see medicine prescription history.
Added more medicines
Added 13 more options for the user to input when entering their symptoms. Added a new function addSymptom to add users' symptoms to the symptoms ArrayList. Added error handling in Patient.java in the event no medication is available for the illness. Removed boilerplate lines in case statement when user inputs their symptoms and moved them to addSymptom Fixed Insomnia.java to display the correct symptom.
Fixed order of @author tags in Javadoc to pass Gradlew checks.
Changed text displaying symptoms in the UserGuide.md after it has been changed to lowercase. Added note under login in UserGuide.md to explain why having multiple passwords for the same username to access different accounts is intentional. Added note under "reporting symptoms" in UserGuide.md in the case that the patient's symptoms does not yield any suitable diagnosis. Added separating line in Information.java when viewing symptom history in viewSymptomHistory(). Tried to fix image not showing in brennanzuz.md.
Fixed outdated output when displaying symptoms in UserGuide.md
Tried to fix image not showing in brennanzuz.md.
Tried to fix image not showing in brennanzuz.md.
Remove typo in PPP
Update project status on PPP
Update about us
Finalise PPP
Formatted PPP
Jeraldchen edit dg
Edit JAR file name in UG
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Doctor Duke is a desktop app for diagnosing patients who have mild illnesses, optimized for use via a Command Line Interface (CLI). If you can type fast, Doctor Duke can diagnose illnesses and dispense medicine accordingly faster than visiting a General Practitioner.